Implement(60881): Added detailed diagnostic for parent to child assignment#61145
Implement(60881): Added detailed diagnostic for parent to child assignment#61145NamHaiBui wants to merge 32 commits intomicrosoft:mainfrom
Conversation
|
@microsoft-github-policy-service agree |
…0881 Syncing with upstream changes
…cMessages.json and changed it's code to 2881
|
This newest change will fail the CI tests. However, it should correctly add the message we are looking for. I didn't accept the baseline so we can perform the difference checker in class tomorrow and ensure the new message is correct. |
typescript-bot
added
the
For Uncommitted Bug
|
This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise. |
typescript-bot
added
For Backlog Bug
NamHaiBui
changed the title
|
This PR contains a bunch of unrelated formatting changes which make this PR hard to review; you should revert these. If you have auto-format enabled in your editor, make sure it's running dprint. |
@jakebailey I've removed the formatting issues. Please let us know if there are any additional changes to be made. |
This reverts commit 8933230.
|
@jakebailey Can you take another look at this issue? I think its ready for review. |
tests/baselines/reference/better_subType_assignable_to_superType_error_messsage.errors.txt
Outdated
Show resolved
Hide resolved
github-project-automation
bot
moved this from Not started
to Waiting on author
in PR Backlog
| conditionalTypes2.ts(15,5): error TS2322: Type 'Covariant<A>' is not assignable to type 'Covariant<B>'. | ||
| Type 'A' is not assignable to type 'B'. | ||
| 'B' could be instantiated with an arbitrary type which could be unrelated to 'A'. | ||
| 'B' is constrained to be a subtype of 'A'. |
There was a problem hiding this comment.
I'm not understanding why this error message is an improvement. What's the smallest example where this error is going to be easier to understand?
There was a problem hiding this comment.
@RyanCavanaugh This error message is an improvement because the original error message says that the two types could be unrelated but we know they are related in a parent child relationship, you just can't assign a parent to a child. We tried to pick a new error message that was similar to the suggestion in the original issue. This is the simplest example we looked at:
function example<SuperType, SubType extends SuperType>(superType: SuperType, subType: SubType) {
subType = superType;
}
|
With 6.0 out as the final release vehicle for this codebase, we're closing all PRs that don't fit the merge criteria for post-6.0 patches. If you think this was a mistake and this PR fits the post-6.0 patch criteria, please post to the 6.0 iteration issue with details (specifically, which PR and which patch criteria it satisfies). Next steps for PRs:
|
8 participants
Fixes #60881